로그 위생 정리 및 레이어별 로깅 규칙 적용#253
Open
ckdals4600 wants to merge 1 commit into
Open
Conversation
ckdals4600
requested review from
Goder-0 and
minibr
and removed request for
minibr
📊 코드 커버리지 리포트
|
ckdals4600
added a commit
that referenced
this pull request
Jun 25, 2026
ckdals4600
force-pushed
the
feature/#252-apply-logging-rule
branch
from
c157df9 to
e9cfe28
Compare
Goder-0
reviewed
Jun 28, 2026
|
|
||
| if (ex != null) { | ||
| log.error("AI 답변 생성 중 오류 발생 - chatId: {}, error: {}", effectiveChatId, ex.getMessage(), ex); | ||
| log.error("AI 답변 생성 중 오류 발생 - chatId={}", chatId, ex); |
Contributor
There was a problem hiding this comment.
PR 본문에는 ChatFacade의 비동기 답변 시작 예외를 4xx/WARN, 5xx/ERROR로 정리했다고 적혀 있는데, 현재 whenComplete 경로는 여전히 예외를 전부 ERROR로 남기고 있습니다.
ckdals4600
force-pushed
the
feature/#252-apply-logging-rule
branch
from
e9cfe28 to
31d2c34
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
관련 이슈
PR 설명
운영 로그에 섞여 있던 민감정보와 정상 흐름 노이즈를 제거하고,
레이어·상황별 로그 레벨을 일관된 규칙으로 정리했습니다.
전 도메인(chat/link/infra/auth/member)의 로깅을 점검했습니다.
적용한 로깅 규칙
변경 사항
민감정보 제거 (우선순위 높음)
SummaryStatusEventListener: 웹소켓 푸시 로그에서 이메일(PII) 제거, INFO → DEBUGLinkService: 링크 생성 로그에서 url(사용자 콘텐츠) 제거로그 레벨 정리
LinkService: update/title/memo 로그 INFO → DEBUG (생성/삭제만 감사 성격으로 유지)SummaryQueue: 큐 적재 로그 INFO → DEBUGS3ImageUploader: 캐시 히트 INFO → DEBUG, S3 업로드 실패 ERROR → WARN(fallback)OgTagCrawler: 크롤링 실패 WARN에서 스택트레이스 제거JwtTokenProvider: 토큰 부재(비로그인 정상 흐름) WARN → DEBUG예외 로깅 정리
ChatFacade: 비동기 답변 시작 예외를 4xx(WARN)/5xx(ERROR)로 구분, 메시지+스택트레이스 중복 전달 제거OAuth2FailureHandler: 공급자 서버 오류(5xx)만 WARN 추가 (사용자 취소 등 4xx는 미기록)불필요한 로깅 제거
MockAnswerClient: 테스트용 목의 사용자 질문 로깅 제거 (@slf4j 함께 제거)